Add MS Teams Notifications via Webhooks#1578
Closed
svenvg93 wants to merge 4 commits intoalexjustesen:develop-0.21.0from
svenvg93:40-add-teams
Closed
Add MS Teams Notifications via Webhooks#1578svenvg93 wants to merge 4 commits intoalexjustesen:develop-0.21.0from svenvg93:40-add-teams
svenvg93 wants to merge 4 commits intoalexjustesen:develop-0.21.0from
svenvg93:40-add-teams
Conversation
Contributor
Author
|
Not sure how this will work when this will happen |
alexjustesen
approved these changes
Jul 18, 2024
Owner
|
Before merging I'll need to do more research if it's worth adding MS Teams if the connector is going to start failing soon. |
Owner
|
@svenvg93 with MS killing off webhook support I propose we don't add MS Teams as a notification channel. If someone is using it I'm also willing to bet its a really small population. |
Contributor
Author
|
@alexjustesen sounds like a good idea. Wouldn’t be worth it to add it now indeed. |
Contributor
Author
|
@alexjustesen I do wanted to have a look to simplify the notifications as some are similar. And maybe optimize the page as well. Let me know what you think |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📃 Description
To cross Teams of the list #40
🪵 Changelog
➕ Added
Add Teams to the notification page
Add Teams to the settings migration
Add a placeholder for the URL
📷 Screenshots
Settings:
Results
